Skip to content

Add LandonSchropp/hunk-verdict and hunk-backup to the extension directory - #1137

Merged
benvinegar merged 2 commits into
modem-dev:mainfrom
LandonSchropp:list-hunk-review-and-hunk-backup
Oct 12, 2026
Merged

benvinegar merged 2 commits into
modem-dev:mainfrom
LandonSchropp:list-hunk-review-and-hunk-backup

Conversation

@LandonSchropp

@LandonSchropp LandonSchropp commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds two hunk-extension-tagged repositories to the community extension directory, inserted alphabetically into EXTENSION_CATALOG in website/src/data/extensions.ts:

  • LandonSchropp/hunk-verdict: Approve, comment on or deny the changes from a modal inside Hunk, and stream each decision to an agent or script with hunk verdict listen.
  • LandonSchropp/hunk-backup: Saves every review's comments as they're written, so closing Hunk never loses them, with hunk backup list and hunk backup show to get them back.

version and apiVersion come from each repository's package.json. Both are tagged (v0.3.0 and v0.1.0). The changeset is empty because this only changes website content.

Test plan

  • Reviewer to confirm the two listings render on the /extensions/ page in a website build.

@vercel

vercel Bot commented Sep 30, 2026

Copy link
Copy Markdown

@LandonSchropp is attempting to deploy a commit to the Modem Team on Vercel.

A member of the Team first needs to authorize it.

@greptile-apps

greptile-apps Bot commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

PR author is not in the allowed authors list.

@benvinegar

Copy link
Copy Markdown
Member

Rebased onto current main and resolved the extension-catalog conflict by preserving both new listings and the existing meldron/hunk-mark-read listing.

Validation: bun run website:check, bun run website:build, and git diff --check origin/main all passed. Both listings appear in the generated extensions page.

One naming concern: Hunk already ships an agent skill named hunk-review, and this extension also documents an installable skill with that name. The extension ID itself does not duplicate an existing directory entry, but the skill name overlaps; could you rename the extension’s skill to distinguish it from Hunk’s bundled review skill?

This comment was generated by Pi using gpt-6.1-sol

@benvinegar
benvinegar force-pushed the list-hunk-review-and-hunk-backup branch from 1c6f6c9 to 9f0cbbb Compare October 11, 2026 20:31
@LandonSchropp LandonSchropp changed the title Add LandonSchropp/hunk-review and hunk-backup to the extension directory Add LandonSchropp/hunk-verdict and hunk-backup to the extension directory Oct 12, 2026
@LandonSchropp

Copy link
Copy Markdown
Contributor Author

@benvinegar Sure thing. I've renamed it to hunk-verdict.

@benvinegar

Copy link
Copy Markdown
Member

Rebased onto the latest main and resolved the extension-directory conflicts. Preserved main’s existing hunk-verdict 0.3.0 listing and added the hunk-backup listing; the redundant verdict update was dropped during the rebase.

Validation: git diff --check origin/main...HEAD and cd website && bun run check passed (0 errors, warnings, or hints).

This comment was generated by Pi using gpt-6.1-sol

@benvinegar
benvinegar force-pushed the list-hunk-review-and-hunk-backup branch from cce2a2b to 6d3ce22 Compare October 12, 2026 15:41
@benvinegar
benvinegar merged commit 745593a into modem-dev:main Oct 12, 2026
13 checks passed
@LandonSchropp
LandonSchropp deleted the list-hunk-review-and-hunk-backup branch October 13, 2026 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants